Thanks! I'll check it maybe in the week.

Hi, @mshr-h san
Thanks for reply.

Those arguments seem optional when linting with slang.
I think they should be in the default of the verilog.linting.slang.arguments section instead of the args because some users don't want to check all the warnings.
We don't need --lint-only option?

I understand your thinking. How about the code below?

• I thought about leaving --single-unit option alone, but we took into account that some people don't want to set it. Since the option is also set when evaluated by sv-test
• Slang has two check phases(lint phase -> elaborate phase). With --lint-only option, Elaborate phase is not checked (only lint phase warnings are checked). I would like the user to be able to decide whether to limit the phase.

+ let args: string[] = [];
- args.push('--single-unit');
+ args.push(`-I ${docFolder}`);
+ args = args.concat(this.includePath.map((path: string) => `-I ${path}`));
+ args.push(this.arguments);
+ args.push(`"${docUri}"`);
- args.push('-Weverything', '--strict-driver-checking'); // Slang all warnings check options
```

@retar-kamuy
Ok. Looks good to me.